The mockMetadataMixin and $tr: key => key mock cause all queries to target translation key names ('altText', 'highContrast') instead of the strings users actually see ('Includes alternative text descriptions for images', 'Includes high contrast text for learners with low vision'). This violates the Kolibri unit testing guide's "Reference translation keys, not hardcoded strings" principle, and contradicts the acceptance criteria requirement to follow Testing Library principles — TL's core guideline is that tests should resemble how users interact with the app.

Neither mock is necessary. translateMetadataString calls metadataStrings.$tr() internally, which is a createTranslator instance that resolves to its _defaultMessages in test environments with no extra setup. Remove both mocks and query by the real translated strings:

import { metadataStrings } from 'shared/strings/metadataStrings';

// Instead of { name: /altText/i }:
screen.getByRole('checkbox', { name: metadataStrings.$tr('altText') });
// → 'Includes alternative text descriptions for images'

screen.getByRole('checkbox', { name: metadataStrings.$tr('highContrast') });
// → 'Includes high contrast text for learners with low vision'

This also means the mocks: { $tr: key => key } block can be removed entirely.

configure({ testIdAttribute: 'data-test' }) should be called at module scope, not inside beforeAll. Jest can initialize parts of the test infrastructure before beforeAll runs, so testId queries that fire during module loading would miss the configuration. The afterAll restore is also risky — if a test throws before afterAll executes, data-testid is never restored, silently breaking other test files that run after this one.

Replace both blocks with a single module-scope call:

configure({ testIdAttribute: 'data-test' });

routes: [] installs VueRouter with no routes. AccessibilityOptions doesn't use routing, so this can be removed.

This smoke test is made redundant by the 'shows document-specific accessibility options' test immediately below, which already renders the same default kind: 'document' case and asserts exactly 3 checkboxes. Consider removing it, or giving it a distinct purpose (e.g. asserting the component renders without throwing for an unknown kind).